-
Notifications
You must be signed in to change notification settings - Fork 823
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #4063 and add select button #4064
base: master
Are you sure you want to change the base?
Fixes #4063 and add select button #4064
Conversation
Works great but you can no longer scroll by dragging the canvas. Not sure what the solution is. Maybe right-mouse button to select? Or long press and drag to select? @pikurasa any suggestions? |
@walterbender I think scrolling canvas by dragging was never possible in MB, it can be scrolled with the help of arrow keys. |
@walterbender @pikurasa I think we should add a new feature in which we can use:
What do you think, should I work on it. |
We also use long press as a way to indicate right-click on devices w/o buttons (or only single buttons). |
@walterbender ok I'll work on it. |
I don't think it is prudent to merge as is since scrolling the block container is pretty fundamental whereas select is not. |
Ok I'll update my changes in this PR only. |
@walterbender I have updated my changes to:
Vedio: scroll-and-select.mp4Please review. |
It is working for me except that when I right-click, I also get the right-click menu. I can still select the region, but it is not a very good UX. |
Also, we need to test on some touch devices. |
I tested and I didn't get right-click menu when I did right click and drag on canvas to select, right click pie menu open only when we do right click on a block (I think that's fundamental). |
I have been testing with Firefox on Fedora. Now I cannot get right click to select. Not sure why. I'll test on Chroma as well. |
Previously I was testing it by launching from pie-issue.mp4you can see I tried reproducing it but It didn't show up. @walterbender could you please tell me which of the changes I made could have caused this? or is this issue existed before my changes? |
I tested on chrome with the local server too. Same problem. Question: Are you in advanced mode? The right-click menu only shows up in advanced mode. Also, it would be nice to treat long press as right click too. |
Maybe a solution might be to add select to the right-hand menu instead of invoking it directly from right click and drag? |
@walterbender yes I was in beginner mode that's why it was not showing. |
@walterbender sounds good, I'll try to implement it. |
@walterbender Rather than adding select to right hand menu, I have added a select button which enable select mode and allows us to select by doing right-click and drag. select.button.mp4How it works:
We can scroll the block container anytime by left-click and drag. |
I think it would be best to put the selector button on the right click menu
to save on some clutter.
…On Sun, Nov 24, 2024, 2:04 PM Harshit Verma ***@***.***> wrote:
@walterbender <https://github.com/walterbender> Rather than adding select
to right hand menu, I have added a select button which enable select mode
and allows us to select by doing right-click and drag.
https://github.com/user-attachments/assets/6926b500-8278-49f7-8978-b2cada0695fc
How it works:
1. When we want to select we have to click select button which enable
the select mode.
2. We can select the blocks as we like by right-click and drag, and
right-hand menu doesn't opens.
3. After selecting we can disable select mode by again clicking the
select button.
4. When select mode is disabled and we are not selecting the
right-hand menu can be opened by right click.
We can scroll the block container anytime by left-click and drag.
@walterbender <https://github.com/walterbender> please test and tell me
your review whether this is fine or not.
I think select button svg needs to be updated as per theme of MB please
help me with that.
—
Reply to this email directly, view it on GitHub
<#4064 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA6PXYMEIDMPZTC7XNEOYOD2CIPKBAVCNFSM6AAAAABR6HQTECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGE3TIMZQGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@walterbender I have put the select button on right-hand menu, this is how it looks select-on-menu.mp4In select mode we can drag and select using either of the mouse button, but here it is preferred to use left-click and drag since it will not open right-menu. |
Looks like a big improvement. But maybe add a notification (textMsg) that selection is enabled (and another message for when it is disabled)? |
@walterbender I have added the textMsg "Select is enabled" and "Select is disabled" Vedio: textMsg.mp4Please review. |
Fixes #4063 this issue arises after the commit b5be318 I have made the required changes to it fix this issue.
@walterbender please review.
Vedio:
fix-scroll.on.select.mp4